Skip to content

feat: Add script count badges to tab navigation #100

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Oct 10, 2025
Merged

feat: Add script count badges to tab navigation #100

merged 2 commits into from
Oct 10, 2025

Conversation

@michelroegl-brunner
Copy link
Member

@michelroegl-brunner michelroegl-brunner commented Oct 10, 2025

Summary

This PR adds script count badges to the tab navigation, showing the number of scripts in each category.

Changes

  • Available Scripts Tab: Shows total count of scripts available from GitHub
  • Downloaded Scripts Tab: Shows count of scripts that have been downloaded locally
  • Installed Scripts Tab: Shows total count of installed script records

Technical Details

  • Added API data fetching hooks to main page component
  • Implemented script count calculation logic for each tab type
  • Added responsive count badges with muted styling
  • Badges update automatically when data changes

UI/UX

  • Count badges use subtle styling that blends with the UI
  • Responsive design works on both desktop and mobile
  • Badges positioned with proper spacing and typography

Testing

  • ✅ TypeScript compilation passes
  • ✅ No linting errors
  • ✅ Responsive design verified
  • ✅ Data fetching and count calculation working correctly

Closes #[issue-number]

@michelroegl-brunner
feat: add script count badges to tab navigation
fc2c58c 
- Add script counts to Available Scripts, Downloaded Scripts, and Installed Scripts tabs
- Counts are calculated from API data and displayed as small badges
- Available scripts count shows total GitHub scripts
- Downloaded scripts count shows scripts that have local versions
- Installed scripts count shows total installed script records
- Badges use muted styling to blend with the UI
@michelroegl-brunner michelroegl-brunner requested a review from a team as a code owner October 10, 2025 10:48
@michelroegl-brunner
fix: use nullish coalescing operator for safer null handling
9517e38 
- Replace logical OR (||) with nullish coalescing (??) for better null/undefined safety
- Fixes ESLint error: @typescript-eslint/prefer-nullish-coalescing
- Ensures build passes successfully
@michelroegl-brunner michelroegl-brunner merged commit c618fef into main Oct 10, 2025
4 checks passed
@michelroegl-brunner michelroegl-brunner deleted the feature/script-count-badges branch October 14, 2025 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@michelroegl-brunner